Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

src/drivers/magnetometer/bosch/bmm350/BMM350.cpp: Publish initial 0 d… #815

Merged
merged 1 commit into from
Nov 18, 2024

Conversation

jlaitine
Copy link

…ata to register mags in module start order

Magnetometer numbers are allocated in the order when the first data gets published to sesors module.

It is problematic if the magnetometers get different IDs on different HW even if the start order of the drivers in the init scripts is the same.

Therefor, publish some dummy data right after successful probe of the driver

@jlaitine jlaitine requested a review from joukkone November 18, 2024 08:46
…hen creating magnetometer driver instance

This ensures that magnetometers instance numbers are allocated in the driver start order.

It is problematic if the magnetometers get different IDs on different HW even if the start order of the
drivers in the init scripts is the same.

Signed-off-by: Jukka Laitinen <[email protected]>
@jlaitine jlaitine force-pushed the fix_mag_start_order branch from 05459a2 to f2e5c23 Compare November 18, 2024 09:16
@jlaitine jlaitine requested a review from joukkone November 18, 2024 12:56
@jlaitine jlaitine merged commit 20ff30c into main Nov 18, 2024
27 checks passed
@jlaitine jlaitine deleted the fix_mag_start_order branch November 18, 2024 14:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants